Skip to content

[LLVM][lit] add system-cygwin feature #152780

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 8, 2025

Conversation

jeremyd2019
Copy link
Contributor

There are a few tests in clang that gate on system-windows but also don't work on Cygwin, so add a system-cygwin feature for them to use.

There are a few tests in clang that gate on system-windows but also
don't work on Cygwin, so add a system-cygwin feature for them to use.
@llvmbot
Copy link
Member

llvmbot commented Aug 8, 2025

@llvm/pr-subscribers-testing-tools

Author: None (jeremyd2019)

Changes

There are a few tests in clang that gate on system-windows but also don't work on Cygwin, so add a system-cygwin feature for them to use.


Full diff: https://github.com/llvm/llvm-project/pull/152780.diff

1 Files Affected:

  • (modified) llvm/utils/lit/lit/llvm/config.py (+2)
diff --git a/llvm/utils/lit/lit/llvm/config.py b/llvm/utils/lit/lit/llvm/config.py
index 649636d4bcf4c..b04fb25f3c030 100644
--- a/llvm/utils/lit/lit/llvm/config.py
+++ b/llvm/utils/lit/lit/llvm/config.py
@@ -107,6 +107,8 @@ def __init__(self, lit_config, config):
             features.add("system-solaris")
         elif platform.system() == "OS/390":
             features.add("system-zos")
+        elif sys.platform == "cygwin":
+            features.add("system-cygwin")
 
         # Native compilation: host arch == default triple arch
         # Both of these values should probably be in every site config (e.g. as

Copy link
Member

@mstorsjo mstorsjo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jeremyd2019 jeremyd2019 merged commit 37bcd93 into llvm:main Aug 8, 2025
12 checks passed
@jeremyd2019 jeremyd2019 deleted the llvm-lit-config-cygwin branch August 8, 2025 20:00
jeremyd2019 added a commit to jeremyd2019/llvm-project that referenced this pull request Aug 9, 2025
There are a few tests in clang that gate on system-windows but also
don't work on Cygwin, so add a system-cygwin feature for them to use.

(cherry picked from commit 37bcd93)
jeremyd2019 added a commit to jeremyd2019/llvm-project that referenced this pull request Aug 9, 2025
There are a few tests in clang that gate on system-windows but also
don't work on Cygwin, so add a system-cygwin feature for them to use.

(cherry picked from commit 37bcd93)
jeremyd2019 added a commit to jeremyd2019/llvm-project that referenced this pull request Aug 12, 2025
There are a few tests in clang that gate on system-windows but also
don't work on Cygwin, so add a system-cygwin feature for them to use.

(cherry picked from commit 37bcd93)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants